The problem I see with this pr, is that if there are say two sockets in the list and the second (last one) is to be closed, we'll mark it as closable but not actually handle it. I would much prefer to see us build a list of them, then delete and remove from the socket list at the end if there are any in list, or somehow otherwise handle this. Otherwise we'll have two code paths where sockets should be closed; one in the loop and one outside the loop which would require some code duplication.

or simply checking if it is the last member before flagging it for close, and if it is just close it.
i.e
if(sock == SockeList.back())
CloseSocket(sock);
else
slclose = sock;

I made the changes, to handle your concerns.

Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE